label: Compute label baselines correctly

When we were switching smallest and widest, we were not switching the
baselines.

Add GtkTreeListRow type to the docs

Add missing listmodel and selection types to the docs

tree list model: Add a missing annotation

The GtkTreeListModelCreateModelFunc returns
a new reference.

inspector: Avoid critical warnings

We were still using this but it's not defined in the UI xml.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/2295

Merge branch 'inspector-display' into 'master'

Inspector display

See merge request GNOME/gtk!1234

inspector: Make the css tree respect inspected display

The display is used here to force reparsing the theme
on the right settings object.

inspector: Make logs respect inspected display

inspector: Make inspect button use inspected display

This may not matter, but just for cleanliness.

inspector: Make general respect inspected display

inspector: Make visual settings respect the inspected display

This is not quite complete, GTK and GSK debug flags
are not per display, and slowdown and text direction
are not either.

inspector: Make css editor respect inspected display

Stop using gdk_display_get_default and use the
inspected display instead.

inspector: Make object tree respect inspected display

Stop using gdk_display_get_dfault and use the
inspected display instead.

window: Get rid of the inspector_window static

Use the new per-display inspector plumbing and
stop relying on a static variable.

inspector: Attach inspector window to display

Give the inspector window a ::inspected-display
property, and pass a display when obtaining an
inspector window. Update the caller.

window: Inspector cleanups

Reduce the use of the inspector_window static.
The goal is to have one inspector window per
display, so we need to get rid of the static.

window: Remove inspector updating fully

Commit 05e752e096722 left some ruins behind.
Clean that up.

inspector: Init yourself

No need to have GTK call this from the outside.

inspector: Cleanup

Remove an unused enum.

Make a test with a custom display

This completely breaks the inspector.

Merge branch 'doc-update' into 'master'

doc: Cleanup doc removing mentions of gdk_surface_shape_combine_region()

See merge request GNOME/gtk!1232

doc: Cleanup doc removing mentions of gdk_surface_shape_combine_region()

Merge branch 'matthiasc/for-master' into 'master'

fix leaks in the testsuite

See merge request GNOME/gtk!1231

Fix treelistmodel tests

They were failing to return a reference where they
need to. This was uncovered by fixing an unrelated
ref leak.

Fix leaks in tests

All the list model tests were leaking items,
because g_list_model_get_item is transfer full.

Fixing these unveils a crash in the treelistmodel
and maplistmodel tests.

Update German translation

builder: Improve error message

builder: Allow objects in gtk_builder_value_from_string_type()

Instead of throwing an error, lookup objects with
gtk_builder_get_object().

gtk-demo: Use normal way to lookup up symbols

We're a normal application, so we can use the normal way to hook up code
into builder files, which is using dlsym() for the function pointers.

There's no need to to extra work exporting static symbols.

css: Handle invalid :not() selectors

We weren't correctly ending the ( ) block when encountering an error.

Testcases added.

Fixes #2281

Merge branch 'wip/wayland-hidpi-swap-region' into 'master'

Fix HiDPI swap regions on Wayland

See merge request GNOME/gtk!1228

wayland/vulkancontext: Fix present region on HiDPI

VkPresentRegionsKHR is expected to operate in buffer coordinates, but
the region we get passed is in surface coordinates, so it must be scaled.

wayland/glcontext: Fix damage region on HiDPI

The passed regions are in surface coordinates, but
eglSwapBuffersWithDamage() is in buffer coordinates.

Merge branch 'fix-treemodelsort_processed_as_treemodel' into 'master'

Annotate GtkTreeModelSort.new_with_model() appropriately

See merge request GNOME/gtk!1134

Annotate GtkTreeModelSort.new_with_model() appropriately

The new_with_model() method is a constructor, and it returns a GtkTreeModelSort
instance, even if the C API returns a GtkTreeModel for the convenience of C
developers.

Fixes: #1077

gl renderer: Implement blurred inset shadow nodes

And with this...

Fixes #1101

gl renderer: Remove unused uniform from inset shadow shader

opbuffer: Remove unused op member

testsuite: Add an inset shadow rendering test case

gl renderer: Cache current scale in RenderOpBuilder

We call ops_get_scale a lot, so this should be better. It will also make
a later x/y split for the scaling easier.

cssshadowvalue: Simplify inset shadow nodes to color nodes if possible

inset shadows like this are frequently used to implement highlights in
CSS without crowing them min size of the element.

gl renderer: Save one level of indentation

window: Don't set invalid csd input shape

This might happen if the CSS values of the decoration node are broken,
e.g. if people *accidentally* type large negative values for the
margins.

Fixes #2268

snapshot: Reorder color matrix nodes containing a transform node

A color matrix node that contains a transform node can also be expressed
the other way around, as a transform node containing a color matrix
node.

In the general case, the color matrix node will have to draw its
child to a texture so it can color every pixel of that texture, but the
renderers can short-cut this if the child of the color matrix node is
already a texture node. So if we have a node tree like

Color Matrix
    - Transform
        - Texture

The renderer would have to either check the grandchild of the color
matrix or simply fall back to rendering the transform node to a texture.

In the new configuration:

Transform
    - Color Matrix
        - Texture

The renderer can easily see that the child node of the color matrix node
is a texture, and skip rendering it to a texture.

This is for example happening in current Adwaita for spinners, which are
rotated symbolics.

snapshot: Factor out color matrix merging

We will use that elsewhere in the next commit.

gl renderer: Implement blurred shadow nodes

gl renderer: Take a TextureRegion in blur_texture

So we can refer to textures on a texture atlas.

gl renderer: Scale blur radius

We scale the texture size, so as a consequence we have to scale the blur
radius.

gl renderer: Add some often needed debug code

gl renderer: Blur shader improvements

Use a two-pass blur shader, fix a few other things and unify the
blurring of blur nodes and blurred outset shadow nodes.

Related to #1283

testblur: Fix default blur radius

sync it up with the adjustment value. also just add the destroyed
handler.

Merge branch 'improve-popover-menu-style' into 'master'

Adwaita: Improve popover.menu styling

See merge request GNOME/gtk!1083

Adwaita: Remove unused popover check/radio styling

Adwaita: Flatten circular buttons in popover.menu

Just reuse the list button style.

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Adwaita: Make check & radio symbolic icon style in popover.menu

The disabled modelbutton color needs to be specified explicitly
in order for check & radio to properly read the color with
currentColor.

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Adwaita: Improve popover.menu styling

Based on the mockup:
https://gitlab.gnome.org/Teams/Design/os-mockups/blob/master/menus/menu-design-patterns.png

- Remove horizontal padding from popover.menu contents
- Remove roundness from modelbutton
- Adjust sizing to match the mockup
- Fix visual glitches on button.image-button.model
- Add missing mergin to arrow icons
- Fix accelerator position in rtl direction
- Fix accelerator color

https://gitlab.gnome.org/GNOME/gtk/issues/1824

Merge branch 'wip/muktupavels/opaque-region-master' into 'master'

window: fix opaque region

See merge request GNOME/gtk!1222

Merge branch 'sass-no-extend-compound-master' into 'master'

themes: remove various @extend with compound selectors

See merge request GNOME/gtk!1223

themes: remove various @extend with compound selectors

They are no longer supported by sass and broken with libsass 3.6.3
(https://github.com/sass/libsass/issues/3033)

This removes them by replacing them with a placeholder selector. This at
least brings the resulting CSS size down a bit so gtk can be build
again.

`%button.flat.suggested-action` has been replaced by
`%selection_mode_button_flat`, which is a more appropriate selector for
`.selection-mode button.titlebutton`.

The CSS was generated with libsass 3.5.5.

Co-authored-by: Christoph Reiter <reiter.christoph@gmail.com>
See https://gitlab.gnome.org/GNOME/gtk/issues/2237

window: fix opaque region

Decoration node for drawing is used only for client side decorated
windows, but corners from opaque region is subtracted also for
normal windows.

Rename function to better reflect what it does and do not subtract
corners if decoration node was not used for drawing.

Merge branch 'gtk-link-color' into 'master'

Adwaita: always use link specific colors for link selector

See merge request GNOME/gtk!1201

Merge branch 'matthiasc/for-master' into 'master'

some builder-tool tests

See merge request GNOME/gtk!1220

Update the test scripts

All tools got renamed to have a gtk4 prefix.

Add a test for the previous fix

Merge branch 'matthiasc/for-master' into 'master'

builder-tool: Fix handling of layout properties

See merge request GNOME/gtk!1219

builder-tool: Fix handling of layout properties

We were not properly identifying layout properties
that need to be kept, causing GtkGrid left-attachment
to go missing in widget-factory.ui.

Fix up widget-factory ui

Merge branch 'dont-color-labels-directly-gtk4' into 'master'

Adwaita: Avoid directly coloring labels wherever possible (GTK4)

Closes #1643

See merge request GNOME/gtk!1209

reftests: Fix css-multi-state test

Properly unset theme CSS and use the right selector for the toggle
buttons.

Adwaita: always use link specific colors for link selector

link target uses `$selected_bg_color` and `$selected_fg_color` for the
definition of some selectors (e.g. selected).

While `$link_color` and `$link_visited_color` are based on
`$selected_bg_color`, this does not seem coherent.

Replacing `$selected_*g_color` in link target style with link colors
defined in `_colors.scss file`.

widget-factory: Fix a popover menu

This one is manually constructed, and things have
changed enough for that to break. Fix it up.

gl renderer: Don't use g_assert() for code that should always run

g_assert can be compiled out.

gl renderer: Don't save repeat nodes to disk

This is just debuggin code someone forgot.

Merge branch 'single-compilation-symbol' into 'master'

Use a single compilation symbol

See merge request GNOME/gtk!1216

Merge branch 'fix-issue-1646-master' into 'master'

HighContrast: Add missing color to entry

Closes #1646

See merge request GNOME/gtk!1215

Use a single compilation symbol

We use a compilation symbol in our build to allow the inclusion of
specific headers while building GTK, to avoid the need to include only
the global header.

Each namespace has its own compilation symbol because we used to have
different libraries, and strict symbol visibility between libraries;
now that we have a single library, and we can use private symbols across
namespaces while building GTK, we should have a single compilation
symbol, and simplify the build rules.

HighContrast: Add missing color to entry

So entries don't inherit color from the parent element anymore.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1646

Merge branch 'broken-annotation' into 'master'

builder: Fix broken annotation

See merge request GNOME/gtk!1213

builder: Fix broken annotation

Missing a ':', was introduced in 0627bba5a0d.

gl renderer: Load flipped-y offscreen vertex data directly

Instead of loading the unflipped version first and then flipping it.
Don't do it in add_render_ops either but only in the function actually
adding the render ops for the nodes, since those frequently have
early-out conditions that don't need the vertex data at all.

testsuite: Add another repeat node test case

gl renderer: Fix repeat node shader once again

gl renderer: Care more about 0×0 offscreen nodes

These never result in a rendering of course, but we must make sure not
to create a 0×0 texture for them, since that will cause problems later
on.

Merge branch 'unsafe-strdup-printf' into 'master'

gtk-demo: Fix incorrect use of g_strdup_printf

See merge request GNOME/gtk!1211

gtk-demo: Fix incorrect use of g_strdup_printf

Was broken in a refactoring in 4a2f313

Adwaita: Avoid directly coloring labels wherever possible

Directly coloring labels can create a lot of unintended behaviors.
Unfortunately we can't avoid directly coloring `label:disabled` because
it can be used individually. This, however, tries to inherit color from
the parent element of labels wherever possible.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1643

testsuite: Fix expected test output

This reverts part of 91497348bf1ea90ef1056bdc8727bbea645c21e3

cssprovider: Don't add empty rulesets

This change was lost during the parser changes, but the newly added test
exposes it.

And fix the expected error, the syntax has changed.

Fix crash in parse_border()

The bug was introduced in commit:
9b7640b8 by Benjamin Otte, 2012-03-26 17:24:02
styleproperty: Make _gtk_style_property_parse_value() return a CssValue

In that commit, `values` changed from `GValue*` to `GtkCssValue**`,
but one `!G_IS_VALUE (&values[8])` was left untouched. As a result,
if `border` shorthand contains anything after color, it might crash,
depending on memory layout.

New test included.

Fixes: #751

Merge branch 'wip/otte/builder' into 'master'

Reorganize various GtkBuilder things

See merge request GNOME/gtk!1204

builder: Make the struct private

We don't want anyone to subclass it - if we want to allow bindings to
extend GtkBuilder, we will be using delegate objects/functions for it.

builder: Remove get_type_from_name() vfunc

builder: Improve signal connecting

- Propagate the error back to the parser, so we get a proper GError
  instead of a g_warning().
- Connect closures by id, don't construct a name from the ids so that
  glib can take it apart again.

builder: Connect signals automatically

gtk_builder_connect_signals() is no longer necessary, because all the
setup that made it necessary to have this extra step is now done
automatically via the closure functions.

reftests: Switch to gtk_builder_set_closure_func()

widget: Change templates to use closure functions

... instead of connect functions.

builder: Add gtk_builder_set_closure_func()

This allows bindings and templates to setup a way to create closures.

widget: Reinstate custom closure func

It turns out it ws used in various places.

builder: Add gtk_builder_create_closure()

This will be the future way to connect signals automatically (and be
used for other things, too).

For now, gtk_builder_connect_signals_default() is ported to use it.

builder: Remove user_data argument from gtk_builder_connect_signals()

This is pretty unused and gets in the way of the next steps.

A potential side effect is that for templates the widget was passed as
the user data argument. If that turns out to be important, we have to
special case that situation.